Skip to content

Conversation

Hounddog
Copy link
Owner

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a80eb61 on single_fixture into 4f09fd9 on master.

@Ocramius
Copy link
Collaborator

Misses tests...

if (is_dir($fixtures)) {
$this->loader->loadFromDirectory($fixtures);
} elseif (file_exists($fixtures)) {
$classes = get_declared_classes();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this class supposed to handle this? Seems to be a case for a context aware fixture loader

@coveralls
Copy link

Coverage Status

Coverage increased (+39.31%) when pulling f8d5008 on single_fixture into 1a3571c on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e5d764f on single_fixture into 1a3571c on master.

@@ -0,0 +1,156 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should not be committed

@Ocramius Ocramius self-assigned this Nov 10, 2014
@coveralls
Copy link

Coverage Status

Coverage increased (+15.31%) to 23.0% when pulling 20c3046 on single_fixture into 1a3571c on master.

*
* @license http://www.opensource.org/licenses/lgpl-license.php LGPL
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to contact all authors of the library if you want to change the license :|

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have the license already set as MIT for the library. Unfortunately it is set differetnly in the files.

@coveralls
Copy link

Coverage Status

Coverage increased (+15.54%) to 23.23% when pulling 6a830d3 on single_fixture into 1a3571c on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants